Skip to content

Added actionable notification when a vendor does not have a subcontracting location code - #9787

Open
6CRIPT wants to merge 7 commits into
mainfrom
bugs/642229-workcenter-does-not-warn
Open

Added actionable notification when a vendor does not have a subcontracting location code#9787
6CRIPT wants to merge 7 commits into
mainfrom
bugs/642229-workcenter-does-not-warn

Conversation

@6CRIPT

@6CRIPT 6CRIPT commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Added so the user can detect the missing subcontracting location code before a future process requires it.

Fixes AB#642229

@6CRIPT
6CRIPT requested a review from a team July 28, 2026 09:48
@6CRIPT 6CRIPT self-assigned this Jul 28, 2026
@github-actions github-actions Bot added the AL: Apps (W1) Add-on apps for W1 label Jul 28, 2026
@6CRIPT
6CRIPT enabled auto-merge July 28, 2026 09:49
@github-actions github-actions Bot added the SCM GitHub request for SCM area label Jul 28, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone Jul 28, 2026
Comment thread src/Apps/W1/Subcontracting/App/src/General/SubcNotificationMgmt.Codeunit.al Outdated
…t.Codeunit.al

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Comment thread src/Apps/W1/Subcontracting/Test/Tests/SubcSubcontractingUITest.Codeunit.al Outdated
Comment thread src/Apps/W1/Subcontracting/Test/Tests/SubcSubcontractingUITest.Codeunit.al Outdated
PredragMaricic
PredragMaricic previously approved these changes Jul 31, 2026

@PredragMaricic PredragMaricic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The notification lifecycle is handled correctly: a prior warning is recalled, empty, invalid, and configured vendors do not warn, and the action opens the vendor from the notification payload. The focused UI tests cover the warning, no-warning, recall, and action paths. No blocking findings.

@github-actions

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 SCM GitHub request for SCM area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants